-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[ADD] Biteship as a delivery provider #14854
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 19.0
Are you sure you want to change the base?
Conversation
f830217 to
de3365d
Compare
de3365d to
4a5bd21
Compare
erjer-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kaju-odoo ! This looks great! I made some suggestions to wording to make things clearer, as well as editing in several places for line length requirements. There were a couple of places where I had clarifying questions, as well. If you have any concerns that you think we can clear up more quickly outside of github, feel free to ping me on discord (erjer.odoo).
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
...ons/inventory_and_mrp/inventory/shipping_receiving/setup_configuration/biteship_shipping.rst
Outdated
Show resolved
Hide resolved
erjer-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kaju-odoo ! This looks great! I made some suggestions to wording to make things clearer, as well as editing in several places for line length requirements. There were a couple of places where I had clarifying questions, as well. If you have any concerns that you think we can clear up more quickly outside of github, feel free to ping me on discord (erjer.odoo).
4a5bd21 to
0cf3ab5
Compare
0cf3ab5 to
188d9ee
Compare
Thanks for the review! Sorry it took a while to respond, I fixed some of the issues you pointed out |

Biteship is a shipping service aggregator that facilitates the integration of Indonesian shipping carriers with Odoo. Once integrated, users can select shipping carriers on inventory operations in their Odoo database.
Task ID : 4641931